Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore!: v21 Namespace & Upgrade Handler #969

Merged
merged 5 commits into from
Mar 2, 2024
Merged

Conversation

joelsmith-2019
Copy link
Contributor

Upgrade namespace to v21 from v19. (v20 was used as a patch for CW VM issues introduced in v19.)

Waiting on #968 before completing.

Reecepbcups and others added 2 commits February 14, 2024 12:56
* fix!: load from the proper data dir

* v20 plain upgrade

* lintor

* execute contract's before and after an upgrade

* remove v20 storekey addition

* CreateV20UpgradeHandler

* remove v20

* fix upgrade test (v18 -> v19)
@joelsmith-2019 joelsmith-2019 self-assigned this Feb 16, 2024
@joelsmith-2019 joelsmith-2019 added the blocked We can't do this until something else is done label Feb 16, 2024
* Bump golangci/golangci-lint-action from 3 to 4

Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 3 to 4.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v3...v4)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* update lint action

* do the easy stuff first....

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dimiandre dimiandre marked this pull request as ready for review March 2, 2024 09:52
@dimiandre dimiandre removed the blocked We can't do this until something else is done label Mar 2, 2024
@dimiandre dimiandre merged commit 00b2a38 into main Mar 2, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants